Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ARROW-17252: [R] Intermittent valgrind failure (wait for thread pool approach) #13779

Closed
wants to merge 9 commits into from

Conversation

paleolimbot
Copy link
Member

No description provided.

@paleolimbot
Copy link
Member Author

@github-actions crossbow submit test-r-linux-valgrind

@github-actions
Copy link

github-actions bot commented Aug 2, 2022

@github-actions
Copy link

github-actions bot commented Aug 2, 2022

⚠️ Ticket has not been started in JIRA, please click 'Start Progress'.

@github-actions
Copy link

github-actions bot commented Aug 2, 2022

Revision: d38781f

Submitted crossbow builds: ursacomputing/crossbow @ actions-c87d0167a9

Task Status
test-r-linux-valgrind Azure

@paleolimbot
Copy link
Member Author

@github-actions crossbow submit test-r-linux-valgrind

@github-actions
Copy link

github-actions bot commented Aug 2, 2022

Revision: aa429ab

Submitted crossbow builds: ursacomputing/crossbow @ actions-66d788420f

Task Status
test-r-linux-valgrind Azure

@paleolimbot
Copy link
Member Author

@github-actions crossbow submit test-r-linux-valgrind

@github-actions
Copy link

github-actions bot commented Aug 3, 2022

Revision: ca0e250

Submitted crossbow builds: ursacomputing/crossbow @ actions-74e7027351

Task Status
test-r-linux-valgrind Azure

@paleolimbot
Copy link
Member Author

@github-actions crossbow submit test-r-linux-valgrind

@github-actions
Copy link

github-actions bot commented Aug 4, 2022

Revision: fcbd862

Submitted crossbow builds: ursacomputing/crossbow @ actions-17da8ef45a

Task Status
test-r-linux-valgrind Azure

@paleolimbot
Copy link
Member Author

@github-actions crossbow submit test-r-linux-valgrind

@github-actions
Copy link

github-actions bot commented Aug 4, 2022

Revision: 6d48ad5

Submitted crossbow builds: ursacomputing/crossbow @ actions-57dbd50f21

Task Status
test-r-linux-valgrind Azure

@paleolimbot
Copy link
Member Author

@github-actions crossbow submit test-r-linux-valgrind

@github-actions
Copy link

github-actions bot commented Aug 5, 2022

Revision: ac58c2d

Submitted crossbow builds: ursacomputing/crossbow @ actions-be43d8f311

Task Status
test-r-linux-valgrind Azure

@paleolimbot
Copy link
Member Author

@github-actions crossbow submit test-r-linux-valgrind

@github-actions
Copy link

github-actions bot commented Aug 6, 2022

Revision: e38cb2c

Submitted crossbow builds: ursacomputing/crossbow @ actions-f396285a0a

Task Status
test-r-linux-valgrind Azure

@paleolimbot
Copy link
Member Author

@github-actions crossbow submit test-r-linux-valgrind

@github-actions
Copy link

github-actions bot commented Aug 7, 2022

Revision: 7717616

Submitted crossbow builds: ursacomputing/crossbow @ actions-8c1b73e0ba

Task Status
test-r-linux-valgrind Azure

@paleolimbot
Copy link
Member Author

@github-actions crossbow submit test-r-linux-valgrind

@github-actions
Copy link

github-actions bot commented Aug 7, 2022

Revision: 123d8bd

Submitted crossbow builds: ursacomputing/crossbow @ actions-d76d1348a1

Task Status
test-r-linux-valgrind Azure

@paleolimbot
Copy link
Member Author

This PR was useful to run crossbow jobs to see if the known leak caused by shutting down the IO thread pool before a plan finishes is a possible cause of the valgrind errors. This does seem to be the case...we observe intermittent leaks that occasionally are a direct leak of an ExecPlan. Closing in favour of #13773, since that version is a more direct rollback of a change that caused the errors to start.

@paleolimbot paleolimbot closed this Aug 8, 2022
@paleolimbot paleolimbot deleted the r-valgrind-3 branch December 9, 2022 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant